Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement custom OVMF firmware #646

Merged
merged 8 commits into from
Jun 26, 2024

Conversation

nesitor
Copy link
Member

@nesitor nesitor commented Jun 21, 2024

Problem: A user cannot specify which OVMF firmware want to use for the confidential instances.

Solution: Use new aleph-message version that includes that data schema and implement it on the qemu confidential resources.
It also implements the extra volume use for qemu instances.

@nesitor nesitor requested review from olethanh and hoh June 21, 2024 15:30
@nesitor nesitor self-assigned this Jun 21, 2024
@olethanh olethanh force-pushed the andres-feature-implement_custom_ovmf branch from 24b83ab to 5dd3fd4 Compare June 25, 2024 14:00
@olethanh
Copy link
Collaborator

I made a few fix after testing, waiting to see if the CI pass now

@olethanh
Copy link
Collaborator

Had to make some more modification to make the volumes working

Copy link

codecov bot commented Jun 25, 2024

Codecov Report

Attention: Patch coverage is 91.33858% with 11 lines in your changes missing coverage. Please review.

Project coverage is 60.23%. Comparing base (a5128f8) to head (60ca2a2).
Report is 28 commits behind head on dev-confidential.

Files Patch % Lines
...aleph/vm/controllers/qemu_confidential/instance.py 57.14% 6 Missing ⚠️
...c/aleph/vm/hypervisors/qemu_confidential/qemuvm.py 50.00% 3 Missing ⚠️
src/aleph/vm/models.py 50.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@                 Coverage Diff                  @@
##           dev-confidential     #646      +/-   ##
====================================================
+ Coverage             59.13%   60.23%   +1.10%     
====================================================
  Files                    65       67       +2     
  Lines                  5778     5787       +9     
  Branches                627      618       -9     
====================================================
+ Hits                   3417     3486      +69     
+ Misses                 2215     2151      -64     
- Partials                146      150       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@olethanh olethanh force-pushed the andres-feature-implement_custom_ovmf branch from f4653a4 to 9f2b601 Compare June 26, 2024 13:42
@olethanh olethanh force-pushed the andres-feature-implement_custom_ovmf branch from 9f2b601 to 544ffce Compare June 26, 2024 14:40
@olethanh olethanh merged commit 97f343a into dev-confidential Jun 26, 2024
22 checks passed
@Psycojoker Psycojoker deleted the andres-feature-implement_custom_ovmf branch July 24, 2024 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants